Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

API: une demande de correction de dossier peut être de type "outdated" #9744

Merged

Conversation

colinux
Copy link
Member

@colinux colinux commented Nov 22, 2023

Première étape pour améliorer plus tard le wording usager quand une correction concerne plutôt une demande de mise à jour/reconfirmation du dossier, plutôt qu'une correction spécifique.

Cf #9485 pour les détails

@colinux colinux requested a review from tchak November 22, 2023 16:13
@colinux colinux force-pushed the correction-reason-update branch from 19598f7 to 2a0bffb Compare November 22, 2023 16:14
"""
Le dossier doit être mis à jour et revalidé
"""
to_update
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Je me demande si outdated ne serai pas une bone option ici et coherent avec les autres noms : incomplete, incorrect, outdated ?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

T'as raison, j'y avais pensé en plus mais je l'avais écarté pour de mauvaises raisons. C'est amendé.

@colinux colinux force-pushed the correction-reason-update branch from 2a0bffb to 249539f Compare November 23, 2023 14:02
@colinux colinux changed the title API: une demande de correction de dossier peut être de type "to_update" API: une demande de correction de dossier peut être de type "outdated" Nov 23, 2023
@colinux colinux added this pull request to the merge queue Nov 23, 2023
Merged via the queue into demarches-simplifiees:main with commit be994c8 Nov 23, 2023
15 checks passed
@colinux colinux deleted the correction-reason-update branch November 23, 2023 17:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants